Skip to content

Comments

[Issue #493] Standardize Checks and CI Across the Project#523

Open
jcrichlake wants to merge 19 commits intomainfrom
493-build-ci-checks
Open

[Issue #493] Standardize Checks and CI Across the Project#523
jcrichlake wants to merge 19 commits intomainfrom
493-build-ci-checks

Conversation

@jcrichlake
Copy link
Collaborator

@jcrichlake jcrichlake commented Feb 20, 2026

Summary

Changes proposed

What was added, updated, or removed in this PR.

The goal of this PR is to standardize the various build scripts across all of the modules in the project. This effort includes changing any python modules to use a makefile and to change any of the typescript projects to leverage pnpm checks that would also be called via makefile

Context for reviewers

Testing instructions, background context, more in-depth details of the implementation, and anything else you'd like to call out or ask reviewers. Explain how the changes were verified.

This PR hopes to reduce build script discrepancies across the different modules in order to expedite future development. After this the available checks across modules should be as uniform as possible.

Additional information

Screenshots, GIF demos, code examples or output to help show the changes working as expected.

Ran any scripts that were updated or added in each of the modules. Below is a list of commands run and verified.

/lib/changelog-emitter

  • pnpm check:lint
  • pnpm check:types
  • pnpm checks
  • pnpm test:coverage
  • pnpm audit

In lib/cli

  • pnpm build
  • pnpm typespec
  • pnpm check:types
  • pnpm checks
  • pnpm test:coverage
  • pnpm audit
    In lib/core
  • pnpm prepare
  • pnpm check:types
  • pnpm checks
  • pnpm audit

In lib/python-sdk

  • make install
  • make format
  • make lint
  • make test
  • make build
  • make check-format
  • make check-lint
  • make check-types
  • make checks

In lib/ts-sdk

  • pnpm check:types
  • pnpm checks
  • pnpm test:coverage
  • pnpm audit

In /website

  • 'pnpm check:types`
  • pnpm checks
  • pnpm test

@github-actions github-actions bot added cli Issue or PR related to the @common-grants/cli library website Issues related to the website core Issues related to @common-grants/core library python Issue or PR related to Python tooling sdk Issue or PR related to our SDKs typescript Issue or PR related to TypeScript tooling py-sdk Related to Python SDK ts-sdk Related to TypeScript SDK labels Feb 20, 2026
@jcrichlake jcrichlake changed the title 493 Standardize Checks and CI Across the Project [Issue #493] Standardize Checks and CI Across the Project Feb 21, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 23, 2026

🚀 Website Preview Deployed!

Preview your changes at: https://cg-pr-523.billy-daly.workers.dev

This preview will be automatically deleted when the PR is closed.

@jcrichlake jcrichlake marked this pull request as ready for review February 23, 2026 20:23
@jcrichlake jcrichlake requested a review from widal001 February 23, 2026 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cli Issue or PR related to the @common-grants/cli library core Issues related to @common-grants/core library py-sdk Related to Python SDK python Issue or PR related to Python tooling sdk Issue or PR related to our SDKs ts-sdk Related to TypeScript SDK typescript Issue or PR related to TypeScript tooling website Issues related to the website

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Dev tools] Standardize checks across sections of the codebase

1 participant